Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix 'no such file or directory error' when --rm-dist and no directory #297

Merged
merged 1 commit into from
Feb 17, 2021

Conversation

k1LoW
Copy link
Owner

@k1LoW k1LoW commented Feb 17, 2021

$ rm -rf sample/mysql
$ ./tbls doc my://root:mypass@localhost:33308/testdb -c testdata/test_tbls.yml -f sample/mysql --rm-dist
open sample/mysql: no such file or directory

@k1LoW k1LoW added the bug Something isn't working label Feb 17, 2021
@k1LoW k1LoW self-assigned this Feb 17, 2021
@codecov
Copy link

codecov bot commented Feb 17, 2021

Codecov Report

Merging #297 (d19b44f) into master (c3cffef) will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #297      +/-   ##
==========================================
- Coverage   66.54%   66.47%   -0.08%     
==========================================
  Files          25       25              
  Lines        4101     4101              
==========================================
- Hits         2729     2726       -3     
- Misses       1089     1091       +2     
- Partials      283      284       +1     
Impacted Files Coverage Δ
datasource/datasource.go 40.14% <0.00%> (-2.12%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3cffef...d19b44f. Read the comment docs.

@k1LoW k1LoW merged commit 3742284 into master Feb 17, 2021
@k1LoW k1LoW deleted the fix-rm-dist-when-no-dir branch February 17, 2021 10:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant